Skip to content

remove outdated op references#1594

Open
roethke wants to merge 4 commits into
masterfrom
roethke/remove-op-refs
Open

remove outdated op references#1594
roethke wants to merge 4 commits into
masterfrom
roethke/remove-op-refs

Conversation

@roethke

@roethke roethke commented Jun 8, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

What changed? Why?
Removes references to OP which are no longer relevant post-Azul.

Node operators:

  • run-a-base-node.mdx — OP_NODE_L1_ETH_RPC/OP_NODE_L1_BEACON → BASE_NODE_L1_ETH_RPC/BASE_NODE_L1_BEACON
  • troubleshooting.mdx — All OP_NODE_* env vars → BASE_NODE_*, all op-node → base-consensus

Network info:

  • network-fees.mdx — Replaced docs.optimism.io/stack/transactions/fees link with Base's own spec; "OP Stack's implementation of EIP-1559" → "Base's own implementation"; Jovian upgrade link docs.optimism.io/notices/upgrade-17 → /base-chain/specs/upgrades/jovian/overview
  • diffs-ethereum-base.mdx — Replaced 5 docs.optimism.io links with equivalent Base docs (bridging, opcodes, address aliasing, fees, finality)
  • troubleshooting-transactions.mdx — Removed broken docs.optimism.io link from DA throttling note
  • block-building.mdx — Vanilla section: removed op-geth binary and dead GitHub link (op-geth no longer supported post-Azul)

API references:

  • eth_getLogs.mdx, flashblocks-api-overview.mdx — "OP Stack extension" → "Base L2 extension"
  • eth_getTransactionByHash.mdx — "OP Stack deposit transactions" → "Base deposit transactions"
  • eth_getTransactionReceipt.mdx — All 6 "OP Stack L2 field" → "Base L2 field"

Specs:

  • derivation.mdx — "(2s on Optimism)" → "(2s on Base)"; op-node → base-consensus in Engine API version section
  • p2p.mdx — op-node → base-consensus in two places
  • precompiles.mdx — Removed dead op-geth implementation link
  • withdrawals.mdx — "Optimism L2" → "Base L2"
  • azul/exec-engine.mdx — "OP Stack system contracts" → "Base system contracts"
  • isthmus/derivation.mdx — mainnet.optimism.io example URL → generic Ethereum L1 RPC placeholder

Notes to reviewers

How has it been tested?
Locally

@cb-heimdall

cb-heimdall commented Jun 8, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@mintlify

mintlify Bot commented Jun 8, 2026

Copy link
Copy Markdown

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
coinbase 🟡 Building – Jun 8, 2026, 4:59 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@mintlify

mintlify Bot commented Jun 8, 2026 •
edited
Loading

Copy link
Copy Markdown

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
base 🟢 Ready View Preview Jun 8, 2026, 5:01 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@roethke roethke requested a review from mbeutt-cb June 8, 2026 17:26
@roethke roethke marked this pull request as ready for review June 8, 2026 21:27
@mbeutt-cb

mbeutt-cb commented Jun 8, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Most of this looks great! One thing I noticed is the block building page still references op-rbuilder instead of the base-builder: https://github.com/base/base/tree/main/crates/builder. The flashblocks architecture page does as well. It's probably worthwhile to do another sweep on keywords of everything we replaced here: https://docs.google.com/document/d/1gWb2fLfAfYJsqkOerndVs2RXOUnVChb4M2Ik2tReavc/edit?tab=t.a1c7tdkvo7f1#heading=h.ksz9k5dxohf7

mbeutt-cb
mbeutt-cb approved these changes Jun 9, 2026
View reviewed changes

@mbeutt-cb mbeutt-cb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These changes look good. Thanks for doing another sweep on the azul work.

@cb-heimdall

cb-heimdall commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

Review Error for mbeutt-cb @ 2026-06-09 22:02:36 UTC
User failed mfa authentication, either user does not exist or public email is not set on your github profile. \ see go/mfa-help

@roethke roethke requested a review from soheimam June 9, 2026 22:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@soheimam soheimam Awaiting requested review from soheimam

1 more reviewer

@mbeutt-cb mbeutt-cb mbeutt-cb approved these changes

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@roethke @cb-heimdall @mbeutt-cb